Greptile Summary

This auto-generated PR adds two new directory token lifecycle events (dsync.token.created / dsync.token.deleted), a new UserOrganizationMembershipBaseWithUser class, Name fields on several user-related entities, an AdminPortal enum value, and switches the ListMembershipsForResource* authorization methods to return the new with-user type (a breaking API change).

• P1 – OrganizationMembership.User declared non-nullable without null test coverage: User User { get; set; } = default! suppresses compiler warnings, but neither the _nulls fixture nor any test covers a response where user is absent. If any existing endpoint omits the field, callers will get a NullReferenceException at runtime.
• P2 – DsyncTokenDeleted.Data typed as DsyncTokenCreatedData: Both token events share the same payload shape today, so this works, but reusing the created-event's data class for the deleted event is confusing and will be a source of churn if the schemas ever diverge.

Confidence Score: 3/5

Safe to merge with low risk if the API guarantees user is always present in OrganizationMembership responses; a null-safety gap warrants verification before merging.

A P1 finding exists around the non-nullable User property on OrganizationMembership with no null-case test coverage, which could cause NullReferenceExceptions at runtime if the API ever omits the field. The P2 naming issue on DsyncTokenDeleted.Data does not affect correctness today.

src/WorkOS.net/Entities/OrganizationMembership.cs and the corresponding organization_membership_nulls.json fixture need attention to confirm or fix null-safety for the new User property.

Important Files Changed

%%{init: {'theme': 'neutral'}}%%
classDiagram
    class OrganizationMembership {
        +string Id
        +string UserId
        +string OrganizationId
        +OrganizationMembershipCreatedDataStatus Status
        +bool DirectoryManaged
        +string? OrganizationName
        +Dictionary CustomAttributes
        +DateTimeOffset CreatedAt
        +DateTimeOffset UpdatedAt
        +AddRolePermission Role
        +User User  NEW non-nullable
    }
    class UserOrganizationMembershipBaseWithUser {
        +string Id
        +string UserId
        +string OrganizationId
        +OrganizationMembershipCreatedDataStatus Status
        +bool DirectoryManaged
        +string? OrganizationName
        +Dictionary CustomAttributes
        +DateTimeOffset CreatedAt
        +DateTimeOffset UpdatedAt
        +User User
    }
    class DsyncTokenCreated {
        +DsyncTokenCreatedData Data NEW
        +EventContext? Context
    }
    class DsyncTokenDeleted {
        +DsyncTokenCreatedData Data NEW reuses Created type
        +EventContext? Context
    }
    class DsyncTokenCreatedData {
        +string Object
        +string Id
        +string DirectoryId
        +string? OrganizationId
        +string TokenSuffix
        +DateTimeOffset CreatedAt
        +DateTimeOffset? ExpiresAt
    }
    class User {
        +string Id
        +string Email
        +string? FirstName
        +string? LastName
    }
    DsyncTokenCreated --> DsyncTokenCreatedData : Data
    DsyncTokenDeleted --> DsyncTokenCreatedData : Data shared
    OrganizationMembership --> User : User non-nullable
    UserOrganizationMembershipBaseWithUser --> User : User

_{Reviews (1): Last reviewed commit: "feat(generated): Add new event types, fi..." | Re-trigger Greptile}